reformat - move paragraph direction and image size from style attribute to "classic" attribute#39
reformat - move paragraph direction and image size from style attribute to "classic" attribute#39ophiry wants to merge 1 commit intoPythonFreeCourse:mainfrom
Conversation
| "metadata": {}, | ||
| "source": [ | ||
| "<img src=\"images/logo.jpg\" style=\"display: block; margin-left: auto; margin-right: auto;\" alt=\"לוגו של מיזם לימוד הפייתון. נחש מצויר בצבעי צהוב וכחול, הנע בין האותיות של שם הקורס: לומדים פייתון. הסלוגן המופיע מעל לשם הקורס הוא מיזם חינמי ללימוד תכנות בעברית.\">" | ||
| "<img alt=\"לוגו של מיזם לימוד הפייתון. נחש מצויר בצבעי צהוב וכחול, הנע בין האותיות של שם הקורס: לומדים פייתון. הסלוגן המופיע מעל לשם הקורס הוא מיזם חינמי ללימוד תכנות בעברית.\" src=\"images/logo.jpg\" style=\"display: block; margin-left: auto; margin-right: auto;\"/>" |
There was a problem hiding this comment.
If it is OK on your side, I'd prefer not to move around attributes in places where it doesn't make a difference :)
There was a problem hiding this comment.
I used a html parser (BeautifulSoup) to do the reformatting - there's no direct control on all the details of the generated html
| "לפעמים נרצה לצרף דברי הסבר לקוד שכתבנו.<br>\n", | ||
| "כדי להוסיף הסברים לצד הקוד נשתמש בתו סולמית, ומייד אחריה נכתוב את ההסברים שרצינו לצרף.<br>\n", | ||
| "<p dir=\"rtl\" style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n", | ||
| "לפעמים נרצה לצרף דברי הסבר לקוד שכתבנו.<br/>\n", |
There was a problem hiding this comment.
Let's not add closing tags to our brs, we're on HTML5 :)
| "<li><code>\"_)(*&^%\"</code></li>\n", | ||
| "<li><code>\"_)('*&^%\"</code></li>\n", | ||
| " <li><code>\"_)(\"*&^%\"</code></li>\n", | ||
| "<li><code>\"_)(*&^%\"</code></li>\n", |
|
I'm a little late to the party, but I noticed you all aren't using a notebook review tool and wanted to invite you to review this pull request with GitNotebooks: https://gitnotebooks.com/PythonFreeCourse/Notebooks/pull/39 It lets you do things like comment on rendered markdown and code cells, so might be an easy win for your PR reviews. |
3 participants
re: #37